[PATCH 1 of 9] MD: possible typo

Patch name: md-possible-typo.patch

Fix a value printed in kiB but labeled as 'blocks'.

Signed-off-by: Jonathan Brassow <jbrassow [at] redhat.com>

Index: linux-2.6/drivers/md/md.c
============================================================ =======
--- linux-2.6.orig/drivers/md/md.c
+++ linux-2.6/drivers/md/md.c
[at] [at] -6867,7 +6867,7 [at] [at] void md_do_sync(mddev_t *mddev)
*/
window = 32*(PAGE_SIZE/512);
printk(KERN_INFO "md: using %dk window, over a total of %llu blocks.\n",
- window/2,(unsigned long long) max_sectors/2);
+ window/2, (unsigned long long) max_sectors);

atomic_set(&mddev->recovery_active, 0);
last_check = 0;
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo [at] vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Jonathan Brassow [ Di, 24 Mai 2011 05:05 ] [ ID #2059961 ]

Re: [PATCH 1 of 9] MD: possible typo

On Mon, 23 May 2011 22:05:58 -0500 Jonathan Brassow <jbrassow [at] f14.redhat.com>
wrote:

> Patch name: md-possible-typo.patch
>
> Fix a value printed in kiB but labeled as 'blocks'.
>
> Signed-off-by: Jonathan Brassow <jbrassow [at] redhat.com>
>
> Index: linux-2.6/drivers/md/md.c
> ============================================================ =======
> --- linux-2.6.orig/drivers/md/md.c
> +++ linux-2.6/drivers/md/md.c
> [at] [at] -6867,7 +6867,7 [at] [at] void md_do_sync(mddev_t *mddev)
> */
> window = 32*(PAGE_SIZE/512);
> printk(KERN_INFO "md: using %dk window, over a total of %llu blocks.\n",
> - window/2,(unsigned long long) max_sectors/2);
> + window/2, (unsigned long long) max_sectors);
>
> atomic_set(&mddev->recovery_active, 0);
> last_check = 0;
> --
> To unsubscribe from this list: send the line "unsubscribe linux-raid" in
> the body of a message to majordomo [at] vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html


"blocks" has traditionally meant "kibibytes" in md nomenclature, so this was
"correct".

I'd be quite happy to change the word "blocks" to "KB" (or even "KiB") though.

NeilBrown
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo [at] vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
NeilBrown [ Mi, 25 Mai 2011 00:18 ] [ ID #2060014 ]
Linux » gmane.linux.raid » [PATCH 1 of 9] MD: possible typo

Vorheriges Thema: [PATCH 6 of 9] MD: no sync IO while suspended
Nächstes Thema: mdadm problem